Skip to content

[update] docs related to export with json2excel/excel2json libraries#62

Open
mafanya23 wants to merge 2 commits into
nextfrom
actualize-export-links-5197
Open

[update] docs related to export with json2excel/excel2json libraries#62
mafanya23 wants to merge 2 commits into
nextfrom
actualize-export-links-5197

Conversation

@mafanya23
Copy link
Copy Markdown
Contributor

No description provided.

@mafanya23 mafanya23 requested a review from SergMitMin May 25, 2026 12:55
@mafanya23 mafanya23 self-assigned this May 25, 2026
…eet docs

- add next (latest) and x.x (specific version) as parallel options with bullet lists
- update code examples to use next as default with inline comment for x.x alternative
- add default CDN URL (next/worker.js?vx) explicitly in exportModulePath config
- add ?vx suffix to all CDN URLs
- remove standalone 'Replace x.x' sentences, fold version note into descriptions
- update react/props.md example and note for both importModulePath and exportModulePath
Comment thread docs/loading_data.md
- use `"../libs/json2excel/x.x/worker.js"` for a specific version (replace `x.x` with the actual version from the [GitHub repository](https://github.com/dhtmlx/json2excel))
- by providing a link to the file from CDN:
- use `"https://cdn.dhtmlx.com/libs/json2excel/next/worker.js?vx"` for the latest version
- use `"https://cdn.dhtmlx.com/libs/json2excel/x.x/worker.js?vx"` for a specific version (replace `x.x` with the actual version from the [GitHub repository](https://github.com/dhtmlx/json2excel))
Copy link
Copy Markdown

@SergMitMin SergMitMin Jun 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

наверное ссылки на версионные файлы в cdn(те которые не next) не нужно давать, оч мало шансов что юзерам понадобится неактуальная версия

в целом(как я понимаю) - ожидаемые сценарии:

  1. юзеры берут next с cdn

  2. или используют экспорт локально, тогда они как раз могут сами указывать версию x.x в пути, чтобы знать какая у них версия лежит

так что в моментах с локальным экспорта - можно писать про x.x, а в моментах с cdn - стоит оставить next

Comment thread docs/loading_data.md
~~~jsx
var spreadsheet = new dhx.Spreadsheet(document.body, {
exportModulePath: "../libs/json2excel/1.0/worker.js"
exportModulePath: "../libs/json2excel/next/worker.js" // latest version; use x.x instead of next for a specific version
Copy link
Copy Markdown

@SergMitMin SergMitMin Jun 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

можно не добавлять комментарий
// latest version; use x.x instead of next for a specific version

Comment thread docs/react/props.md
Comment on lines +172 to +173
importModulePath="../libs/excel2json/next/worker.js" // latest version; use x.x instead of next for a specific version
exportModulePath="../libs/json2excel/next/worker.js" // latest version; use x.x instead of next for a specific version
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

можно не добавлять комментарии
// latest version; use x.x instead of next for a specific version

- use `"../libs/json2excel/x.x/worker.js"` for a specific version (replace `x.x` with the actual version from the [GitHub repository](https://github.com/dhtmlx/json2excel))
- by providing a link to the file from CDN:
- use `"https://cdn.dhtmlx.com/libs/json2excel/next/worker.js?vx"` for the latest version
- use `"https://cdn.dhtmlx.com/libs/json2excel/x.x/worker.js?vx"` for a specific version (replace `x.x` with the actual version from the [GitHub repository](https://github.com/dhtmlx/json2excel))
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

для cdn можно не указывать эту часть (актуально ко всем cdn фрагментам):
- use "https://cdn.dhtmlx.com/libs/json2excel/x.x/worker.js?vx" for a specific version (replace x.x with the actual version from the GitHub repository)

~~~jsx {2}
const spreadsheet = new dhx.Spreadsheet("spreadsheet", {
exportModulePath: "../libs/json2excel/1.0/worker.js",
exportModulePath: "../libs/json2excel/next/worker.js", // latest version; use x.x instead of next for a specific version
Copy link
Copy Markdown

@SergMitMin SergMitMin Jun 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

комментарий можно убрать
// latest version; use x.x instead of next for a specific version

- by providing a link to the file from CDN: `"https://cdn.dhtmlx.com/libs/json2excel/1.0/worker.js"`

By default the link to CDN is used.
- by providing a local path to the file on your computer:
Copy link
Copy Markdown

@SergMitMin SergMitMin Jun 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По поводу фрагмента - by providing a local path to the file on your computer

не оч корректная формулировка и лучше использовать тож самое что и в Suite
For a local export server, set the path to the worker.js file relative to your server:

актуально ко всем местам где оно встречается

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants